Skip to content
This repository was archived by the owner on Jul 1, 2025. It is now read-only.

Conversation

compnerd
Copy link
Contributor

No description provided.

@nadavrot
Copy link
Contributor

@compnerd I like the change to add a default type to get handle.

I think that we can make DefaultFloat == float all over the code. When I started working on this I assumed that we would need this flexibility, but now I see that everything is wired for the case where FloatTy = float. When we load numbers from caffe models they are assumed to be float. So, I suggest just getting rid of DefaultFloat and making it equal to float. ok?

Rather than explicitly specifying the floating point precision
everywhere, assume the default unless otherwise specified.
Remove DefaultFloatTy and inline the default typedef into all current
uses.
@compnerd
Copy link
Contributor Author

Sounds reasonable enough to me. I can remove the use of the DefaultFloatTy. I wonder if we can de-templateize some of the declarations for a code-size win as well.

@nadavrot
Copy link
Contributor

@compnerd Which declarations?

@compnerd
Copy link
Contributor Author

Some of the handle returning types were templated IIRC, but I didn't track them closely. I think we could do that as a follow up.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants